chore(nox): chore update noxfile with format session that uses ruff#16647
chore(nox): chore update noxfile with format session that uses ruff#16647chalmerlowe wants to merge 4 commits intomainfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request replaces isort and black with ruff for code formatting and import sorting across multiple packages. The review identifies several issues, including missing tool installations, missing session implementations, and the use of undefined variables. It also recommends removing redundant formatting sessions, restoring type hints, and refactoring duplicated path lists to improve maintainability.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
Unit test failures are in sqlalchemy-spanner. |
|
@gemini-code-assist |
|
@chalmerlowe Thank you for reaching out. Based on the changes introduced in this pull request, which primarily involve updating The modifications are confined to the build and formatting tooling, not the functional code of the Without access to the specific unit test logs or error messages, it's impossible for me to pinpoint the exact cause of the failures. However, if the I recommend investigating the |
Partial fix for #16014
Provides the following update to multiple noxfiles:
formatnox session to use ruff for formattingWhat it does NOT do:
This is not a universal panacea... it is purposefully kept lean and focuses on a single task.
It does not delete black versions, it does not delete isort versions, etc.